Skip to content

Add testing_buildBlockV1#9901

Draft
marcindsobczak wants to merge 18 commits intomasterfrom
feature/testing_build_block
Draft

Add testing_buildBlockV1#9901
marcindsobczak wants to merge 18 commits intomasterfrom
feature/testing_build_block

Conversation

@marcindsobczak
Copy link
Contributor

@marcindsobczak marcindsobczak commented Dec 8, 2025

Implementing ethereum/execution-apis#710

Changes

  • PoC implementation of a new rpc endpoint testing_buildBlockV1

Types of changes

What types of changes does your code introduce?

  • Bugfix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a change that causes existing functionality not to work as expected)
  • Optimization
  • Refactoring
  • Documentation update
  • Build-related changes
  • Other: Description

Testing

Requires testing

  • Yes
  • No

If yes, did you write tests?

  • Yes
  • No

@LukaszRozmej
Copy link
Member

LukaszRozmej commented Feb 23, 2026

What is the status of it compared to ethereum/execution-apis#747 ?

@marioevz
Copy link

marioevz commented Feb 23, 2026

I'm testing this in EELS, using execute command, and noticed that the call fails with:

JSONRPCError(code=-32602, message=Missing parameter does not have a default value. (Parameter 'parameters'), data=System.ArgumentException: Missing parameter does not have a default value. (Parameter 'parameters')
E      at System.Reflection.MethodBase.HandleTypeMissing(ParameterInfo paramInfo, RuntimeType sigType)
E      at System.Reflection.MethodBaseInvoker.InvokeWithManyArgs(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
E      at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
E      at Nethermind.JsonRpc.JsonRpcService.ExecuteAsync(JsonRpcRequest request, String methodName, ResolvedMethodInfo method, JsonRpcContext context) in /nethermind/src/Nethermind/Nethermind.JsonRpc/JsonRpcService.cs:line 131)

if the last parameter is missing (extra data), however the spec mentions that required: false for this last parameter.

@marcindsobczak
fix extraData nullability
10fa4d6
@marcindsobczak
Merge branch 'master' into feature/testing_build_block
a13592e 
# Conflicts:
#	src/Nethermind/Nethermind.JsonRpc/Modules/ModuleType.cs
#	src/Nethermind/Nethermind.Merge.Plugin.Test/MergePluginTests.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LukaszRozmej LukaszRozmej Awaiting requested review from LukaszRozmej LukaszRozmej will be requested when the pull request is marked ready for review LukaszRozmej is a code owner

@benaadams benaadams Awaiting requested review from benaadams benaadams will be requested when the pull request is marked ready for review benaadams is a code owner

@asdacap asdacap Awaiting requested review from asdacap asdacap will be requested when the pull request is marked ready for review asdacap is a code owner

@flcl42 flcl42 Awaiting requested review from flcl42 flcl42 will be requested when the pull request is marked ready for review flcl42 is a code owner

@damian-orzechowski damian-orzechowski Awaiting requested review from damian-orzechowski damian-orzechowski will be requested when the pull request is marked ready for review damian-orzechowski is a code owner

@MarekM25 MarekM25 Awaiting requested review from MarekM25 MarekM25 will be requested when the pull request is marked ready for review MarekM25 is a code owner

@Marchhill Marchhill Awaiting requested review from Marchhill Marchhill will be requested when the pull request is marked ready for review Marchhill is a code owner

@smartprogrammer93 smartprogrammer93 Awaiting requested review from smartprogrammer93 smartprogrammer93 will be requested when the pull request is marked ready for review smartprogrammer93 is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

new feature rpc trie

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@marcindsobczak @LukaszRozmej @marioevz @kamilchodola